Fix docker & Makefile perms by chrispsommers · Pull Request #202 · sonic-net/DASH

chrispsommers · 2022-08-24T00:24:35Z

NOTE: prior make all's will leave content in your workspace which can't be cleaned using the modified makefiles, so please make clean before upgrading to this. If necessary, use sudo rm -rf <dangling content> to clean up manually.

Remove all sudo commands from Makefiles except those which use Linux netdevs during SW packet tests.
More selective use of chmod commands (w/o sudo) during make, to grant access to running containers to modify the user workspace via volume mounts
Refactor all Docker images to run as user dashuser:dashuser instead of original Docerfile author's identify (moi).
Change CI script to run as normal CI user (runner) except for SW packet tests, to replicate the developer experience
Test for veth's before deleting them to avoid superfluous deletes which require sudo

Update p4c-bmv2 dockerfile to use p4lang/p4c:latest + fix perms.

chrispsommers · 2022-08-26T01:44:23Z

Reviewers, please clone the dev branch and see if you can build and run tests. Thanks!

mukeshmv

lgtm.
Worked for me. Faced some permission issues with the first make clean on existing workspace. But that was most probably my bad since I should have cleaned first before switching to this branch. But subsequent repeated make clean and make all went through fine. Followed by successful PTF test run.

chrispsommers · 2022-08-26T17:31:55Z

lgtm. Worked for me. Faced some permission issues with the first make clean on existing workspace. But that was most probably my bad since I should have cleaned first before switching to this branch. But subsequent repeated make clean and make all went through fine. Followed by successful PTF test run.

@mukeshmv Thanks for trying! I had the same issue myself just today, I'll add comments to the PR.

vmytnykx · 2022-09-06T10:37:05Z

@chrispsommers it works for me. Just one note probably not related to this changes:

Had to modify test/third-party/traffic_gen/[un]deploy_ixiac.sh scripts to use docker compose instead of docker-compose because of docker-compose: command not found error. I think we might need to move those scripts to new docker compose tool instead of deprecated docker-compose.

chrispsommers · 2022-09-06T18:17:37Z

@chrispsommers it works for me. Just one note probably not related to this changes:

Had to modify test/third-party/traffic_gen/[un]deploy_ixiac.sh scripts to use docker compose instead of docker-compose because of docker-compose: command not found error. I think we might need to move those scripts to new docker compose tool instead of deprecated docker-compose.

@vmytnykx Thanks for the review! re: docker- compose vs. docker compose - I wasn't aware of the new command. I'll look into reconciling this. Meanwhile, did you follow the instructions here? I'm not disagreeing that we should update to use docker compose but I want to make sure there's no functional problems with the existing workflow. Thanks.

vmytnykx · 2022-09-07T09:16:43Z

@chrispsommers it works for me. Just one note probably not related to this changes:
Had to modify test/third-party/traffic_gen/[un]deploy_ixiac.sh scripts to use docker compose instead of docker-compose because of docker-compose: command not found error. I think we might need to move those scripts to new docker compose tool instead of deprecated docker-compose.

@vmytnykx Thanks for the review! re: docker- compose vs. docker compose - I wasn't aware of the new command. I'll look into reconciling this. Meanwhile, did you follow the instructions here? I'm not disagreeing that we should update to use docker compose but I want to make sure there's no functional problems with the existing workflow. Thanks.

Yes, it works also. But I think in the future we will need to avoid deprecated docker- compose. For now changes are good.

KrisNey-MSFT · 2022-09-07T16:07:17Z

Merge 201, then this one

KrisNey-MSFT · 2022-09-11T20:36:42Z

Per Community conversations and PR reviews, merging.

* AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]>

* AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]>

@lguohan

commit 9700152 Merge: 32332b5 d003f4e Author: Michael Miele <[email protected]> Date: Sat Sep 17 16:43:45 2022 -0700 Merge branch 'Azure:main' into main commit d003f4e Author: Chris Sommers <[email protected]> Date: Fri Sep 16 22:43:03 2022 -0700 Upgrade to ixia-c 0.0.1-3383 (sonic-net#226) * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch. Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.) * Update ixia-c traffic engine to match controller. * Update snappi version to match ixia-c-components. Co-authored-by: Chris Sommers <[email protected]> commit 543df5c Author: Ze Gan <[email protected]> Date: Fri Sep 16 01:36:01 2022 +0800 [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. commit 94b0168 Author: Chris Sommers <[email protected]> Date: Wed Sep 14 10:27:22 2022 -0700 Slide deck for today's SAI Challenger presentation commit 10056d7 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:33:36 2022 -0700 Update .wordlist.txt commit f1c5245 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:31:16 2022 -0700 Fix Spelling (sonic-net#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (sonic-net#210) * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> commit 8694184 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:25:46 2022 -0700 Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. commit 2ce121e Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:37:40 2022 -0700 Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> commit e31982c Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:36:47 2022 -0700 Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> commit 32332b5 Merge: 9f62724 6d1c50a Author: Michael Miele <[email protected]> Date: Wed Sep 7 17:46:27 2022 -0700 Merge branch 'Azure:main' into main commit 9f62724 Merge: 9b51bb7 e5e461a Author: Michael Miele <[email protected]> Date: Thu Aug 25 09:11:30 2022 -0700 Merge branch 'Azure:main' into main commit 9b51bb7 Merge: 54eca5a 3c15d95 Author: Michael Miele <[email protected]> Date: Mon Jul 18 18:40:31 2022 -0700 Merge branch 'Azure:main' into main commit 54eca5a Merge: 30c65ab 7b5ce5e Author: Michael Miele <[email protected]> Date: Tue Jun 28 12:10:45 2022 -0700 Merge branch 'Azure:main' into main commit 30c65ab Merge: 48ae39c c9612ff Author: Michael Miele <[email protected]> Date: Mon Jun 20 10:27:44 2022 -0700 Merge branch 'Azure:main' into main commit 48ae39c Merge: e4b3b64 49d9f2c Author: Michael Miele <[email protected]> Date: Fri Jun 17 10:53:03 2022 -0700 Merge branch 'Azure:main' into main commit e4b3b64 Merge: 4120634 231212b Author: Michael Miele <[email protected]> Date: Tue Jun 14 13:39:40 2022 -0700 Merge branch 'Azure:main' into main commit 4120634 Merge: b5c1700 a9ec96a Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:41:12 2022 -0700 Merge branch 'Azure:main' into main commit b5c1700 Merge: f541d44 2e28aef Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:31:01 2022 -0700 Merge branch 'Azure:main' into main commit f541d44 Merge: 1927911 8433203 Author: Michael Miele <[email protected]> Date: Wed Jun 8 13:40:39 2022 -0700 Merge branch 'Azure:main' into main commit 1927911 Merge: 569c32a d0807c8 Author: Michael Miele <[email protected]> Date: Tue May 31 13:49:13 2022 -0700 Merge branch 'Azure:main' into main commit 569c32a Merge: d956391 e420737 Author: Michael Miele <[email protected]> Date: Fri May 20 16:25:33 2022 -0700 Merge branch 'Azure:main' into main commit d956391 Author: KrisNey-MSFT <[email protected]> Date: Tue May 10 16:34:33 2022 -0700 Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images commit 18f88a4 Merge: b2c18a7 221c401 Author: Michael Miele <[email protected]> Date: Mon May 9 08:48:29 2022 -0700 Merge branch 'Azure:main' into main commit b2c18a7 Author: Michael Miele <[email protected]> Date: Mon May 9 08:47:53 2022 -0700 sync-with-azure-dash (#8) * dash-holistic-view (sonic-net#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (#3) Merged in the PR: #2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]>

@lguohan

* sync-with-azure-dash (#8) * dash-holistic-view (#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images * Reorganized dataplane folder. * Fixed links * Reorganized gnmi folder. * Reorganized encrypt-gw-service and express-route-service folders * Reorganized general folder * Fixed link * Reorganized high-avail folder. * Reorganized load-bal-service folder * Reorganized sai folder. * Reorganized stpl-service folder. * Reorganized vnet-peering-service folder. * Reorganized vnet-to-vnet-service folder. * Fixed reference links. * Fixed links to point to the reorganized folders. * Update vnet-to-vnet-service.md Updated metadata * Squashed commit of the following: commit 9700152 Merge: 32332b5 d003f4e Author: Michael Miele <[email protected]> Date: Sat Sep 17 16:43:45 2022 -0700 Merge branch 'Azure:main' into main commit d003f4e Author: Chris Sommers <[email protected]> Date: Fri Sep 16 22:43:03 2022 -0700 Upgrade to ixia-c 0.0.1-3383 (#226) * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch. Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.) * Update ixia-c traffic engine to match controller. * Update snappi version to match ixia-c-components. Co-authored-by: Chris Sommers <[email protected]> commit 543df5c Author: Ze Gan <[email protected]> Date: Fri Sep 16 01:36:01 2022 +0800 [doc]: Fix the typo of TCP UDP protocol number (#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. commit 94b0168 Author: Chris Sommers <[email protected]> Date: Wed Sep 14 10:27:22 2022 -0700 Slide deck for today's SAI Challenger presentation commit 10056d7 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:33:36 2022 -0700 Update .wordlist.txt commit f1c5245 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:31:16 2022 -0700 Fix Spelling (#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (#210) * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> commit 8694184 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:25:46 2022 -0700 Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. commit 2ce121e Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:37:40 2022 -0700 Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> commit e31982c Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:36:47 2022 -0700 Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> commit 32332b5 Merge: 9f62724 6d1c50a Author: Michael Miele <[email protected]> Date: Wed Sep 7 17:46:27 2022 -0700 Merge branch 'Azure:main' into main commit 9f62724 Merge: 9b51bb7 e5e461a Author: Michael Miele <[email protected]> Date: Thu Aug 25 09:11:30 2022 -0700 Merge branch 'Azure:main' into main commit 9b51bb7 Merge: 54eca5a 3c15d95 Author: Michael Miele <[email protected]> Date: Mon Jul 18 18:40:31 2022 -0700 Merge branch 'Azure:main' into main commit 54eca5a Merge: 30c65ab 7b5ce5e Author: Michael Miele <[email protected]> Date: Tue Jun 28 12:10:45 2022 -0700 Merge branch 'Azure:main' into main commit 30c65ab Merge: 48ae39c c9612ff Author: Michael Miele <[email protected]> Date: Mon Jun 20 10:27:44 2022 -0700 Merge branch 'Azure:main' into main commit 48ae39c Merge: e4b3b64 49d9f2c Author: Michael Miele <[email protected]> Date: Fri Jun 17 10:53:03 2022 -0700 Merge branch 'Azure:main' into main commit e4b3b64 Merge: 4120634 231212b Author: Michael Miele <[email protected]> Date: Tue Jun 14 13:39:40 2022 -0700 Merge branch 'Azure:main' into main commit 4120634 Merge: b5c1700 a9ec96a Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:41:12 2022 -0700 Merge branch 'Azure:main' into main commit b5c1700 Merge: f541d44 2e28aef Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:31:01 2022 -0700 Merge branch 'Azure:main' into main commit f541d44 Merge: 1927911 8433203 Author: Michael Miele <[email protected]> Date: Wed Jun 8 13:40:39 2022 -0700 Merge branch 'Azure:main' into main commit 1927911 Merge: 569c32a d0807c8 Author: Michael Miele <[email protected]> Date: Tue May 31 13:49:13 2022 -0700 Merge branch 'Azure:main' into main commit 569c32a Merge: d956391 e420737 Author: Michael Miele <[email protected]> Date: Fri May 20 16:25:33 2022 -0700 Merge branch 'Azure:main' into main commit d956391 Author: KrisNey-MSFT <[email protected]> Date: Tue May 10 16:34:33 2022 -0700 Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images commit 18f88a4 Merge: b2c18a7 221c401 Author: Michael Miele <[email protected]> Date: Mon May 9 08:48:29 2022 -0700 Merge branch 'Azure:main' into main commit b2c18a7 Author: Michael Miele <[email protected]> Date: Mon May 9 08:47:53 2022 -0700 sync-with-azure-dash (#8) * dash-holistic-view (#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update vnet-to-vnet-service.md Updated metadata. * Update vnet-to-vnet-service.md Update date * Update vnet-to-vnet-service.md Changed date * Update vnet-to-vnet-service.md Fixed date * Update vnet-to-vnet-service.md Updated * Update vnet-to-vnet-service.md Fixed date * Create dash-sonic-hld.md Sync with main * Create program-scale-testing-requirements-draft.md Sync with main * Create AMD-Pensando_HA_Proposal.md Sync with main * Update vnet-to-vnet-service.md Update * Delete dash-sonic-hld.md Reorganized general folder. * Delete program-scale-testing-requirements-draft.md Reorganized general folder. * Update README.md Removed the section "Organization of Design & Requirements Documents" because it is obsolete. * Delete AMD-Pensando_HA_Proposal.md Removed design folder. * Fixed broken links * Fixed links and removed obsolete one. * Removed obsolete links. * Update README.md Removed obsolete link. * Update README.md Removed obsolete links. * Removed obsolete links Co-authored-by: KrisNey-MSFT <[email protected]>

* Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]>

@lguohan

* sync-with-azure-dash (sonic-net#8) * dash-holistic-view (sonic-net#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (sonic-net#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images * Reorganized dataplane folder. * Fixed links * Reorganized gnmi folder. * Reorganized encrypt-gw-service and express-route-service folders * Reorganized general folder * Fixed link * Reorganized high-avail folder. * Reorganized load-bal-service folder * Reorganized sai folder. * Reorganized stpl-service folder. * Reorganized vnet-peering-service folder. * Reorganized vnet-to-vnet-service folder. * Fixed reference links. * Fixed links to point to the reorganized folders. * Update vnet-to-vnet-service.md Updated metadata * Squashed commit of the following: commit 9700152 Merge: 32332b5 d003f4e Author: Michael Miele <[email protected]> Date: Sat Sep 17 16:43:45 2022 -0700 Merge branch 'Azure:main' into main commit d003f4e Author: Chris Sommers <[email protected]> Date: Fri Sep 16 22:43:03 2022 -0700 Upgrade to ixia-c 0.0.1-3383 (sonic-net#226) * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch. Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.) * Update ixia-c traffic engine to match controller. * Update snappi version to match ixia-c-components. Co-authored-by: Chris Sommers <[email protected]> commit 543df5c Author: Ze Gan <[email protected]> Date: Fri Sep 16 01:36:01 2022 +0800 [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. commit 94b0168 Author: Chris Sommers <[email protected]> Date: Wed Sep 14 10:27:22 2022 -0700 Slide deck for today's SAI Challenger presentation commit 10056d7 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:33:36 2022 -0700 Update .wordlist.txt commit f1c5245 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:31:16 2022 -0700 Fix Spelling (sonic-net#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (sonic-net#210) * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> commit 8694184 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:25:46 2022 -0700 Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. commit 2ce121e Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:37:40 2022 -0700 Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> commit e31982c Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:36:47 2022 -0700 Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> commit 32332b5 Merge: 9f62724 6d1c50a Author: Michael Miele <[email protected]> Date: Wed Sep 7 17:46:27 2022 -0700 Merge branch 'Azure:main' into main commit 9f62724 Merge: 9b51bb7 e5e461a Author: Michael Miele <[email protected]> Date: Thu Aug 25 09:11:30 2022 -0700 Merge branch 'Azure:main' into main commit 9b51bb7 Merge: 54eca5a 3c15d95 Author: Michael Miele <[email protected]> Date: Mon Jul 18 18:40:31 2022 -0700 Merge branch 'Azure:main' into main commit 54eca5a Merge: 30c65ab 7b5ce5e Author: Michael Miele <[email protected]> Date: Tue Jun 28 12:10:45 2022 -0700 Merge branch 'Azure:main' into main commit 30c65ab Merge: 48ae39c c9612ff Author: Michael Miele <[email protected]> Date: Mon Jun 20 10:27:44 2022 -0700 Merge branch 'Azure:main' into main commit 48ae39c Merge: e4b3b64 49d9f2c Author: Michael Miele <[email protected]> Date: Fri Jun 17 10:53:03 2022 -0700 Merge branch 'Azure:main' into main commit e4b3b64 Merge: 4120634 231212b Author: Michael Miele <[email protected]> Date: Tue Jun 14 13:39:40 2022 -0700 Merge branch 'Azure:main' into main commit 4120634 Merge: b5c1700 a9ec96a Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:41:12 2022 -0700 Merge branch 'Azure:main' into main commit b5c1700 Merge: f541d44 2e28aef Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:31:01 2022 -0700 Merge branch 'Azure:main' into main commit f541d44 Merge: 1927911 8433203 Author: Michael Miele <[email protected]> Date: Wed Jun 8 13:40:39 2022 -0700 Merge branch 'Azure:main' into main commit 1927911 Merge: 569c32a d0807c8 Author: Michael Miele <[email protected]> Date: Tue May 31 13:49:13 2022 -0700 Merge branch 'Azure:main' into main commit 569c32a Merge: d956391 e420737 Author: Michael Miele <[email protected]> Date: Fri May 20 16:25:33 2022 -0700 Merge branch 'Azure:main' into main commit d956391 Author: KrisNey-MSFT <[email protected]> Date: Tue May 10 16:34:33 2022 -0700 Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images commit 18f88a4 Merge: b2c18a7 221c401 Author: Michael Miele <[email protected]> Date: Mon May 9 08:48:29 2022 -0700 Merge branch 'Azure:main' into main commit b2c18a7 Author: Michael Miele <[email protected]> Date: Mon May 9 08:47:53 2022 -0700 sync-with-azure-dash (sonic-net#8) * dash-holistic-view (sonic-net#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (sonic-net#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update vnet-to-vnet-service.md Updated metadata. * Update vnet-to-vnet-service.md Update date * Update vnet-to-vnet-service.md Changed date * Update vnet-to-vnet-service.md Fixed date * Update vnet-to-vnet-service.md Updated * Update vnet-to-vnet-service.md Fixed date * Create dash-sonic-hld.md Sync with main * Create program-scale-testing-requirements-draft.md Sync with main * Create AMD-Pensando_HA_Proposal.md Sync with main * Update vnet-to-vnet-service.md Update * Delete dash-sonic-hld.md Reorganized general folder. * Delete program-scale-testing-requirements-draft.md Reorganized general folder. * Update README.md Removed the section "Organization of Design & Requirements Documents" because it is obsolete. * Delete AMD-Pensando_HA_Proposal.md Removed design folder. * Fixed broken links * Fixed links and removed obsolete one. * Removed obsolete links. * Update README.md Removed obsolete link. * Update README.md Removed obsolete links. * Removed obsolete links Co-authored-by: KrisNey-MSFT <[email protected]>

@lguohan

* Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <[email protected]> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <[email protected]> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <[email protected]> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Incremental progress publishing dockers to ACR (#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (#210) * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Fixes per review feedback #225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Spellcheck Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]>

@lguohan

…ion branches (#272) * Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <[email protected]> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <[email protected]> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <[email protected]> * Added SAI-Challenger to CI setup. 1. Added SAI-Challenger submodule. 2. Added SAI-Challenger basic test. 3. Changed Makefile to build/start/stop SAI-Challenger. Signed-off-by: Maksym Hedeon <[email protected]> * Added saithrift to SAI-Challenger client docker image - Added saithrift to SAI-Challenger client docker image - Changed SAI-C submodule branch Signed-off-by: Maksym Hedeon <[email protected]> * Update example vnet test. 1. Added SAI-Challenger submodule. 2. Added SAI-Challenger basic test. 3. Changed Makefile to build/start/stop SAI-Challenger. Signed-off-by: Kostiantyn Goloveshko <[email protected]> Co-authored-by: Maksym Hedeon <[email protected]> * Add vnet outbound test based on SAI description. Signed-off-by: Maksym Prytoliuk <[email protected]> * Fixup double network host usage for sai-challenger-client Signed-off-by: Konstantin Goloveshko <[email protected]> * Kdt 17: Update docker environment (#8) * Added cgyang submodule * Updated docker env for thrift tests * Makefile refactor Signed-off-by: Maksym Hedeon <[email protected]> * Fixup VNET inbound cleanup removals in test config. Signed-off-by: Konstantin Goloveshko <[email protected]> * Update CA_TO_PA entry in outbound test Signed-off-by: Maksym Prytoliuk <[email protected]> * Add vnet scenarios in DASH config format. * Added dash-style VNET inbound routing test * Added dash-style VNET outbound routing test * Read config from json (temp workaround) * Added JSON setup configs for testbed Signed-off-by: Anton Putria <[email protected]> Co-authored-by: Maksym Prytoliuk <[email protected]> * Fixed saigen links. Signed-off-by: Anton Putria <[email protected]> * Fixes in test scanario. Signed-off-by: Anton Putria <[email protected]> * Fixed path to saigen. Fixed inbound test. Signed-off-by: Anton Putria <[email protected]> * Update submodule Signed-off-by: Anton Putria <[email protected]> * Final test fixes to align with submodue version. Signed-off-by: Anton Putria <[email protected]> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Incremental progress publishing dockers to ACR (#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix #205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Fixed VNET outbound test scenario. Signed-off-by: Anton Putria <[email protected]> * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Outbound test passes with traffic Fixed configuration based on changes in main. Signed-off-by: Maksym Prytoliuk <[email protected]> * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (#210) * SAI apigen support for tables with no action parameters and a single action (#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Update documentation - Added dash-test-sai-challenger.md doc file under test/docs - Updated Makefile make run-saic-test-thrift target with passing parameters. Signed-off-by: Maksym Hedeon <[email protected]> * Updated SAI-Challenger submodule. Signed-off-by: Anton Putria <[email protected]> * Improvements to SAI-Challenger docker flows. - Use docker hub to pull images - Fixed make run-saic-tests to do not show errors - Updated manuals Signed-off-by: Anton Putria <[email protected]> * Fixes per review feedback #225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Update dash-test-sai-challenger.md Signed-off-by: Mircea Dan Gheorghe <[email protected]> * Add bmv2 SAI port attributes (num_active, port_list), dflt vlan, dflt vrf) * Added SAI-Challenger docker build verification to GitHub actions - Added SAI-Challenger docker verification action. - Fixed spellchecker and docker build issues. Signed-off-by: Anton Putria <[email protected]> * Add default .1Q bridge to bmv2 Signed-off-by: Maksym Prytoliuk <[email protected]> * Fixed SAI-Challenger user guide and file namings. Signed-off-by: Anton Putria <[email protected]> * Spellcheck * Scaled VNET outbound test using snappi and saigen. (1st edition) - Split scale and simple tests / jsons - Updated ixia controller version - Added dash_helper for traffic profile scaling. Signed-off-by: Maksym Hedeon <[email protected]> * spellcheck * Move saichallenger client docker to ACR. * Fix .yml formatting. * Fix dependency in action file. * Add missing .env file * remove space in action script, add .env dependencies * fix include path * Add saichalleneger tests to run-all-tests and CI * Fix docker image name. * Correct mixup in image names. * Remove stray hyphen * Run saichallenger workflows on any branch PR. * Added few more VNET scenario examples. - Extended VNET traffic scenarios. - Added docstrings for helper methods. - Removed unused code. Signed-off-by: Anton Putria <[email protected]> Co-authored-by: Maksym Hedeon <[email protected]> * Slides from DASH WG Meeting (SAI Challenger Episode 2) (#254) * Add outbound diagram Signed-off-by: Maksym Prytoliuk <[email protected]> * Added inbound to xfail test results. (#44) - Marked VNET Inbound Routing with xfail - Added pytest.ini with marks definitions. - Decreased duration of tests. Signed-off-by: Anton Putria <[email protected]> * Scaling test configuration documentation - Added scaling approach documentation. - Vnet2vnet traffic scaling scenario description. - Added docstring to the test cases. - Added more test examples. Signed-off-by: Anton Putria <[email protected]> Co-authored-by: Maksym Hedeon <[email protected]> Co-authored-by: Maksym Prytoliuk <[email protected]> * Submodule update and rename * Rename and update the SAI-Challenger submodule * Enabled additional tests based on the fixes in the SAI-Challenger. * Renamed test folder. Signed-off-by: Anton Putria <[email protected]> * PTF: add basic test infra and more test cases (#231) * ptf: add vnet sample test cases for dash - Common VNet API for all tests - Test cases: - Inbound VNI/ENI - Inbound Routing PA validate - Inbound Routing VNI match - Inbound Routing PA SRC IP - Outbound Routing Vnet direct - Outbound Routing direct - CT (just placeholder) - Route (basic) Signed-off-by: Volodymyr Mytnyk <[email protected]> Signed-off-by: Yuriy Harhas <[email protected]> * ptf: run only bmv2 ready test case on CI Signed-off-by: Volodymyr Mytnyk <[email protected]> * ptf: test_saithrift_vnet: removed unused libs Signed-off-by: Volodymyr Mytnyk <[email protected]> * test-cases folder restructure. - created two subfolders in root: functional and scale - removed outdated bmv2_model folder - moved all existing content of test-cases to test-cases/scale Signed-off-by: Anton Putria <[email protected]> * Moved saidashvnet.py to a proper location. Signed-off-by: Anton Putria <[email protected]> * Fixed ENI creation in Outbound tests Signed-off-by: Yuriy Harhas <[email protected]> * Spellchecker related fixes. Signed-off-by: Anton Putria <[email protected]> * Updated VNET test cases based on the PR #238. Signed-off-by: Yuriy Harhas <[email protected]> Signed-off-by: Volodymyr Mytnyk <[email protected]> Signed-off-by: Yuriy Harhas <[email protected]> Signed-off-by: Anton Putria <[email protected]> Co-authored-by: Anton Putria <[email protected]> Co-authored-by: Yuriy Harhas <[email protected]> * Adding updated license files (#266) * Fixbug: SAI Server crash on DASH ACL API (#264) 1. Fix duplicate SAI ID 2. Wrong search action 3. Double delete entry if SAI write error 4. Wrong format of error message Signed-off-by: Ze Gan <[email protected]> * Add VNET v6 PTF test case; Fix v4 PTF test case to address test hang issue (#263) This PR adds VNET v6 test case and also fixes the PTF thrift client issue (the v6 test case just hangs) #237. This is a joint work with @chrispsommers and @aputriax. * Fixed links in the README after renaming folders. (#262) Signed-off-by: Anton Putria <[email protected]> Signed-off-by: Anton Putria <[email protected]> * Correct CI file, was publishing wrong image. Some prior merge must have caused a mixup. Signed-off-by: Maksym Hedeon <[email protected]> Signed-off-by: Kostiantyn Goloveshko <[email protected]> Signed-off-by: Maksym Prytoliuk <[email protected]> Signed-off-by: Konstantin Goloveshko <[email protected]> Signed-off-by: Anton Putria <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Signed-off-by: Mircea Dan Gheorghe <[email protected]> Signed-off-by: Volodymyr Mytnyk <[email protected]> Signed-off-by: Yuriy Harhas <[email protected]> Signed-off-by: Anton Putria <[email protected]> Signed-off-by: Ze Gan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Maksym Hedeon <[email protected]> Co-authored-by: Kostiantyn Goloveshko <[email protected]> Co-authored-by: Maksym Hedeon <[email protected]> Co-authored-by: Maksym Prytoliuk <[email protected]> Co-authored-by: Anton Putria <[email protected]> Co-authored-by: Maksym Prytoliuk <[email protected]> Co-authored-by: Anton Putria <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]> Co-authored-by: MirceaDan <[email protected]> Co-authored-by: Volodymyr Mytnyk <[email protected]> Co-authored-by: Anton Putria <[email protected]> Co-authored-by: Yuriy Harhas <[email protected]> Co-authored-by: Mohammad Hanif <[email protected]>

@lguohan

* sync-with-azure-dash (sonic-net#8) * dash-holistic-view (sonic-net#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images * Reorganized dataplane folder. * Fixed links * Reorganized gnmi folder. * Reorganized encrypt-gw-service and express-route-service folders * Reorganized general folder * Fixed link * Reorganized high-avail folder. * Reorganized load-bal-service folder * Reorganized sai folder. * Reorganized stpl-service folder. * Reorganized vnet-peering-service folder. * Reorganized vnet-to-vnet-service folder. * Fixed reference links. * Fixed links to point to the reorganized folders. * Update vnet-to-vnet-service.md Updated metadata * Squashed commit of the following: commit 9700152 Merge: 32332b5 d003f4e Author: Michael Miele <[email protected]> Date: Sat Sep 17 16:43:45 2022 -0700 Merge branch 'Azure:main' into main commit d003f4e Author: Chris Sommers <[email protected]> Date: Fri Sep 16 22:43:03 2022 -0700 Upgrade to ixia-c 0.0.1-3383 (sonic-net#226) * Upgrade to ixia-c 0.0.1-3383 which fixes hairpin mode and adds option to disable IPv6 at launch. Remove ./disable_veth_ipv6.sh from makefile after deploy-ixiac sincve it's not needed. (Keep it as prerequisite to running test-cases since we still need to prevent linux ICMP6 packets during tests.) * Update ixia-c traffic engine to match controller. * Update snappi version to match ixia-c-components. Co-authored-by: Chris Sommers <[email protected]> commit 543df5c Author: Ze Gan <[email protected]> Date: Fri Sep 16 01:36:01 2022 +0800 [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. commit 94b0168 Author: Chris Sommers <[email protected]> Date: Wed Sep 14 10:27:22 2022 -0700 Slide deck for today's SAI Challenger presentation commit 10056d7 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:33:36 2022 -0700 Update .wordlist.txt commit f1c5245 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:31:16 2022 -0700 Fix Spelling (sonic-net#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (sonic-net#210) * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> commit 8694184 Author: KrisNey-MSFT <[email protected]> Date: Mon Sep 12 16:25:46 2022 -0700 Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. commit 2ce121e Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:37:40 2022 -0700 Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> commit e31982c Author: Chris Sommers <[email protected]> Date: Sun Sep 11 13:36:47 2022 -0700 Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> commit 32332b5 Merge: 9f62724 6d1c50a Author: Michael Miele <[email protected]> Date: Wed Sep 7 17:46:27 2022 -0700 Merge branch 'Azure:main' into main commit 9f62724 Merge: 9b51bb7 e5e461a Author: Michael Miele <[email protected]> Date: Thu Aug 25 09:11:30 2022 -0700 Merge branch 'Azure:main' into main commit 9b51bb7 Merge: 54eca5a 3c15d95 Author: Michael Miele <[email protected]> Date: Mon Jul 18 18:40:31 2022 -0700 Merge branch 'Azure:main' into main commit 54eca5a Merge: 30c65ab 7b5ce5e Author: Michael Miele <[email protected]> Date: Tue Jun 28 12:10:45 2022 -0700 Merge branch 'Azure:main' into main commit 30c65ab Merge: 48ae39c c9612ff Author: Michael Miele <[email protected]> Date: Mon Jun 20 10:27:44 2022 -0700 Merge branch 'Azure:main' into main commit 48ae39c Merge: e4b3b64 49d9f2c Author: Michael Miele <[email protected]> Date: Fri Jun 17 10:53:03 2022 -0700 Merge branch 'Azure:main' into main commit e4b3b64 Merge: 4120634 231212b Author: Michael Miele <[email protected]> Date: Tue Jun 14 13:39:40 2022 -0700 Merge branch 'Azure:main' into main commit 4120634 Merge: b5c1700 a9ec96a Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:41:12 2022 -0700 Merge branch 'Azure:main' into main commit b5c1700 Merge: f541d44 2e28aef Author: Michael Miele <[email protected]> Date: Mon Jun 13 09:31:01 2022 -0700 Merge branch 'Azure:main' into main commit f541d44 Merge: 1927911 8433203 Author: Michael Miele <[email protected]> Date: Wed Jun 8 13:40:39 2022 -0700 Merge branch 'Azure:main' into main commit 1927911 Merge: 569c32a d0807c8 Author: Michael Miele <[email protected]> Date: Tue May 31 13:49:13 2022 -0700 Merge branch 'Azure:main' into main commit 569c32a Merge: d956391 e420737 Author: Michael Miele <[email protected]> Date: Fri May 20 16:25:33 2022 -0700 Merge branch 'Azure:main' into main commit d956391 Author: KrisNey-MSFT <[email protected]> Date: Tue May 10 16:34:33 2022 -0700 Create sdn-packet-transforms-vm-to-vm.svg Edited Fast Path and Slow Path images commit 18f88a4 Merge: b2c18a7 221c401 Author: Michael Miele <[email protected]> Date: Mon May 9 08:48:29 2022 -0700 Merge branch 'Azure:main' into main commit b2c18a7 Author: Michael Miele <[email protected]> Date: Mon May 9 08:47:53 2022 -0700 sync-with-azure-dash (sonic-net#8) * dash-holistic-view (sonic-net#104) * draft document * Updated content and images * added image and updated content * fixed image * Update dash-architecture.md * Fixed image and caption * Update dash-architecture.md * added/reorganized images * Update dash-architecture.md * fixed image and content * Update dash-architecture.md * Update dash-hld-ecosystem.svg * Renamed file. * Update dash-hld-ecosystem.svg * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-hld-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * reorganized content * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed images and content based on feedback * move images, added captions, fixed content * Update dash-hld-project-ecosystem.svg * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md Fixed the "SONiC integration" section and linked to Prince's doc "dash-sonic-hld.md". * Update dash-architecture-view.md * fixed card opacity * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Fixed image * fixed image * fixed image and related content * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md (#3) Merged in the PR: mmiele#2. * Renamed article and added content. * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * Update dash-architecture-view.md * renamed file * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * fixed formating * Update dash-holistic-view.md * Update dash-holistic-view.md * Update dash-holistic-view.md * Fixed images * Fixed TOC * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update documentation/general/design/dash-holistic-view.md Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Apply suggestions from code review Co-authored-by: KrisNey-MSFT <[email protected]> * Updated content and added note * Fixed content * fixed metadata * Created .gitignore and removed bkp files * Fixed content based on feedback. * Fixed content based on feedback. Added links and special notes. Fixed image. * Fixed diagrams: using gNMI container instead of DASH container. * Fixed title and typos * Added reference links * Update documentation/general/design/dash-holistic-view.md Change committed. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-smart-switch.svg Changed "DASH container" name to "gNMI container". * Fixed image and moved `.gitignore` to the main folder. * Apply suggestions from code review Committed Kristina's suggestions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-holistic-view.md Renamed the section API to "SAI headers and behavioral model". Added links to the related README files. * Removed old dash-high-level-design.md deleted dash-high-level-design.md renamed dash-holistic-view.md to dash-high-level-design.md * Added the list of words to generate the "word cloud". * Update README.md Reversed the link order and fixed descriptions. Co-authored-by: KrisNey-MSFT <[email protected]> * Update dash-high-level-design.md * Update dash-high-level-design.md * Update dash-hld-project-ecosystem.svg * Update dash-high-level-design.md Co-authored-by: KrisNey-MSFT <[email protected]> * Update vnet-to-vnet-service.md Updated metadata. * Update vnet-to-vnet-service.md Update date * Update vnet-to-vnet-service.md Changed date * Update vnet-to-vnet-service.md Fixed date * Update vnet-to-vnet-service.md Updated * Update vnet-to-vnet-service.md Fixed date * Create dash-sonic-hld.md Sync with main * Create program-scale-testing-requirements-draft.md Sync with main * Create AMD-Pensando_HA_Proposal.md Sync with main * Update vnet-to-vnet-service.md Update * Delete dash-sonic-hld.md Reorganized general folder. * Delete program-scale-testing-requirements-draft.md Reorganized general folder. * Update README.md Removed the section "Organization of Design & Requirements Documents" because it is obsolete. * Delete AMD-Pensando_HA_Proposal.md Removed design folder. * Fixed broken links * Fixed links and removed obsolete one. * Removed obsolete links. * Update README.md Removed obsolete link. * Update README.md Removed obsolete links. * Removed obsolete links Co-authored-by: KrisNey-MSFT <[email protected]>

@lguohan

* Create dash-docker-p4c-bmv2-acr.yml * Update Makefile * Experimenting with ACR publish workflows Revert p4c-bmv2 tag * Update and rename dash-docker-p4c-bmv2-acr.yml to dash-p4c-bmv2-docker-acr.yml * Experimental ACR publish; change regname Use sonicdash ACR for p4c-bmv2. No publish yet. * Update dash-p4c-bmv2-docker-acr.yml Publish to ACR using secrets * Fix macro errors * Update dash-p4c-bmv2-docker-acr.yml Force trigger. * Uncomment docker push command * Build dash-grpc:1.43.2 docker and publish to ACR * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Publish docker images to ACR and use those in build process. (sonic-net#208) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). Co-authored-by: Chris Sommers <[email protected]> * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Incremental progress - publish to ACRs (sonic-net#212) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). Co-authored-by: Chris Sommers <[email protected]> * Call docker publish make targets from all CI scripts which publish. * Incremental progress publishing dockers to ACR (sonic-net#213) * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. Co-authored-by: Chris Sommers <[email protected]> * Spellcheck - add to wordslist * Sync to main via intermediate dev branch. (sonic-net#216) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Incremental progress publishing dockers to ACR (sonic-net#219) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). * Incremental progress publishing dockers to ACR. (sonic-net#220) * AMD-Pensando HA Proposal document * Add CI script for saithrift-bldr-docker * Remove superfluous ACR publish steps from regular docker build CI script. * Add CI scripts to build and publish remaining docker images to Azure Container Registery (ACR). * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Use make target to publish to ACR (just bmv2-bldr as first candidate). * Call docker publish make targets from all CI scripts which publish. * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Spellcheck - add to wordslist * Fix docker & Makefile perms (sonic-net#202) * Incremental work on docker permissions fixes. * More progress on docker and make permissions issues. * Trial remove -u root from CI file. * Makefile and CI file perms * Fix CI docker flags * Final purge of obs sudo commands to fix perms. * Test for veth's before deleting. Co-authored-by: Chris Sommers <[email protected]> * Fix ipv6 packet noise (sonic-net#211) * Ensure ipv6 disabled on veth's prior to PTF tests, fix sonic-net#205. * CHange docker registrey for ixia-c to Google cloud (was Dockerhub). No functional change but new b uilds will pull new image into workspaces. * Move ipv6 disable code to a separate shell script. Call it after deploy-ixiac target. Co-authored-by: Chris Sommers <[email protected]> * Remove publish steps from non-ACR CI scripts (consequence of merge). Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Trial of env var to define docker img * Add docker .env variable to CI triggers. * Fix stray conflict markers in Makefile. * Fix CI triggers. * Use .env files to define docker images. Fix CI triggers. Remove some docker image CI triggers. * add missing path * Fix path. * Trivial change to force a CI build. * Change Makefile to force CI rebuild. * Trivial change to docker image .env file to test CI triggers. * Change .env file to force CI. * Change grpc docker .env file to trigger CI rebuild. * Change saithrift-bldr docker .env to force CI rebuild. * Change saithrift-client .env file to force CI rebuild/ * Trial docker build/conditional publish. * Get env variable into context. * Correct env filename * Debugging env vars & docker save * Fix CI syntax * Add Makefile targets to upload/download docker image tarball artifact. * Revert to simple conditional job for build+publish. * Conditional CI for non-publish docker job. * Add conditionals to all docker build/publish workflows: run build-only in forks, run build-publish in main repo. * Documenting Docker workflows. * Spellcheck fixes. * Spellcheck fixes. * Add screen cap of skipped job. * Spellcheck * Update PR from dev branch (sync to main) (sonic-net#224) * Update program-scale-testing-requirements-draft.md Update Scale #'s per Keysight/MSFT call; document is a version which needs an update. * Fix Spelling (sonic-net#221) * Update AMD-Pensando_HA_Proposal.md Explicitly add 'parallel' to state synchronization stages (per @lguohan ) * Update AMD-Pensando_HA_Proposal.md (sonic-net#210) * SAI apigen support for tables with no action parameters and a single action (sonic-net#207) * Update .wordlist.txt * Update AMD-Pensando_HA_Proposal.md * Doc dash as submodule (sonic-net#203) * Document third-party workflows using DASH as a Git submodule. * Add URL to sample project. * Incorporate review feedback (typo; missing file). * Spellcheck fixes. * Spellcheck * Add .wordlist.txt to CI triggers. * Spellcheck wordlist. Co-authored-by: Chris Sommers <[email protected]> * Split SAI API (sonic-net#201) Make APIs compatible with SONiC bulk infra Signed-off-by: Marian Pritsak <[email protected]> * Add APP_DB to SAI mapping (sonic-net#102) * Add APP_DB to SAI mapping * Update .wordlist.txt Updating w/Chris Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> * Update .wordlist.txt * Slide deck for today's SAI Challenger presentation * [doc]: Fix the typo of TCP UDP protocol number (sonic-net#223) According to: https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml, the ptotocol number of TCP should be 6, and the UDP's should be 17. * Proofreading, image clarity. Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]> * Use env variable for DASH_ACR_LOGIN_SERVER instead of secret. * Use literal expression instead of env variable. * Use literal expression for ACR repository instead of secret. * Clarify docker publishing. * Spellcheck. * Document another dockerfile workflow. * spellcheck * CI triggers on PR to any branch, not just main. * Fixes per review feedback sonic-net#225 (comment): - fix image; (docxkerhub->ACR in one case - replace "local" with "host" Expand on instructions for writing and running PTF tests (from verbal feedback in WG meeting, not an issue or PR review). Discovered an oversight in the dockerfile .env files which prevented overriding image macros (had to replace = with ?=); tested fix. * Spellcheck. * Spellcheck. * Spellcheck Signed-off-by: Marian Pritsak <[email protected]> Co-authored-by: Chris Sommers <[email protected]> Co-authored-by: Sanjay Thyamagundalu <[email protected]> Co-authored-by: Mukesh Moopath Velayudhan <[email protected]> Co-authored-by: KrisNey-MSFT <[email protected]> Co-authored-by: Marian Pritsak <[email protected]> Co-authored-by: Ze Gan <[email protected]>

chrispsommers added 6 commits August 19, 2022 16:08

Incremental work on docker permissions fixes.

561d473

More progress on docker and make permissions issues.

64b18e4

Trial remove -u root from CI file.

8030814

Makefile and CI file perms

3b0a6cc

Fix CI docker flags

c78b95a

Final purge of obs sudo commands to fix perms.

edc7350

chrispsommers requested a review from marian-pritsak August 24, 2022 00:24

chrispsommers added 2 commits August 25, 2022 18:13

Merge remote-tracking branch 'origin/main' into fix-docker-perms

14e0569

Merge remote-tracking branch 'upstream/main' into fix-docker-perms

4c132b2

Update p4c-bmv2 dockerfile to use p4lang/p4c:latest + fix perms.

chrispsommers marked this pull request as ready for review August 26, 2022 01:42

chrispsommers requested review from mukeshmv and vmytnykx August 26, 2022 01:42

Test for veth's before deleting.

d74fa35

mukeshmv approved these changes Aug 26, 2022

View reviewed changes

chrispsommers mentioned this pull request Aug 30, 2022

make run-all-tests fails (sending with wrong VIP and verifying drop FAILS) #205

Closed

chrispsommers requested a review from mhanif September 1, 2022 17:11

vmytnykx approved these changes Sep 6, 2022

View reviewed changes

marian-pritsak approved these changes Sep 7, 2022

View reviewed changes

Merge remote-tracking branch 'upstream/main' into fix-docker-perms

2499dad

KrisNey-MSFT merged commit e31982c into sonic-net:main Sep 11, 2022

This was referenced Sep 11, 2022

Remove sudo chmod hack from Makefile #179

Closed

make run-all-tests fails #218

Open

chrispsommers deleted the fix-docker-perms branch September 30, 2022 00:54

kcudnik mentioned this pull request Aug 27, 2023

Not optimal build process for DASH repo, SAI and saithrift #427

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix docker & Makefile perms#202

Fix docker & Makefile perms#202
KrisNey-MSFT merged 10 commits intosonic-net:mainfrom
chrispsommers:fix-docker-perms

chrispsommers commented Aug 24, 2022 •

edited

Loading

Uh oh!

chrispsommers commented Aug 26, 2022

Uh oh!

mukeshmv left a comment

Uh oh!

chrispsommers commented Aug 26, 2022

Uh oh!

vmytnykx commented Sep 6, 2022

Uh oh!

chrispsommers commented Sep 6, 2022

Uh oh!

vmytnykx commented Sep 7, 2022

Uh oh!

KrisNey-MSFT commented Sep 7, 2022

Uh oh!

KrisNey-MSFT commented Sep 11, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

chrispsommers commented Aug 24, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chrispsommers commented Aug 26, 2022

Uh oh!

mukeshmv left a comment

Choose a reason for hiding this comment

Uh oh!

chrispsommers commented Aug 26, 2022

Uh oh!

vmytnykx commented Sep 6, 2022

Uh oh!

chrispsommers commented Sep 6, 2022

Uh oh!

vmytnykx commented Sep 7, 2022

Uh oh!

KrisNey-MSFT commented Sep 7, 2022

Uh oh!

KrisNey-MSFT commented Sep 11, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

chrispsommers commented Aug 24, 2022 •

edited

Loading